M10.0 v1.7.5 reorder entropy for uncertainty #75
Conversation
| return self.codes.detailed.both_confirmed | ||
| return self.codes.detailed.ia_confirmed_only | ||
| if uni_overlayed: | ||
| return self.codes.detailed.db_overlayed_only | ||
| return self.codes.detailed.both_unsure |
There was a problem hiding this comment.
On ne passe dans cette ligne (return self.codes.detailed.both_unsure) que si on est à "not high_entropy", je suggère de la passer dans le bloc "if" correspondant, en ligne 365.
Eventuellement en plus, en traitant le cas "high entropy" en premier et en finissant sur le bloc "if not high_entropy" il est possible de supprimer la ligne du "if", puisqu'on est sorti de la méthode en cas de "high entrpoy" just avant. Cela fait une indentation de moins dans un bloc qui en a beaucoup (c'est juste une question de lisibilité)
There was a problem hiding this comment.
On ne passe dans cette ligne (return self.codes.detailed.both_unsure) que si on est à "not high_entropy", je suggère de la passer dans le bloc "if" correspondant, en ligne 365.
On y passe si on n'est pas high entropy, mais on laisse tout de même la possibilité de passer par la BDUni avant, donc impossible de mettre ce return dans le premier bloc.
C'est un peu complexe en l'était mais je ne vois pas vraiment comment simpliifer, à part en se passant totalement de l'entropie. Ca ne me semble pas forcément souhaitable, puisque lorsqu'elle a été ajoutée c'est qu'elle conduisait à une amélioration...
| return self.codes.detailed.both_confirmed | ||
| return self.codes.detailed.ia_confirmed_only | ||
| if uni_overlayed: | ||
| return self.codes.detailed.db_overlayed_only | ||
| return self.codes.detailed.both_unsure |
There was a problem hiding this comment.
On ne passe dans cette ligne (return self.codes.detailed.both_unsure) que si on est à "not high_entropy", je suggère de la passer dans le bloc "if" correspondant, en ligne 365.
On y passe si on n'est pas high entropy, mais on laisse tout de même la possibilité de passer par la BDUni avant, donc impossible de mettre ce return dans le premier bloc.
C'est un peu complexe en l'était mais je ne vois pas vraiment comment simpliifer, à part en se passant totalement de l'entropie. Ca ne me semble pas forcément souhaitable, puisque lorsqu'elle a été ajoutée c'est qu'elle conduisait à une amélioration...
CharlesGaydon
deleted the
M10.0-V1.7.5-Reorder-Entropy-for-Uncertainty
branch
On autorise la BDUni a rattraper une situation de haute entropie. On augmente le range possible des seuils d'entropie dans l'optimisation pour prendre en compte le fait qu'on travaille avec 7 classes prédites.